various #236
various #236
Conversation
|
Good to merge, mostly DocBlock changes and some code changes that don't affect any behaviour. 👍 |
| * Conversation Object | ||
| * | ||
| * @var Longman\TelegramBot\Conversation | ||
| * @var \Longman\TelegramBot\Conversation |
There was a problem hiding this comment.
@MBoretto Did you deliberately leave all full namespaces as opposed to simplified ones?
PhpStorm automatically prefers simplified ones for me.
@akalongman @jacklul
Asking you guys too, which standard shall we go by?
@var \Longman\TelegramBot\Conversationor
@var Conversation(this obviously implies that we have a use Longman\TelegramBot\Conversation at the top.
|
I add it because with the \ in the front and the full namespace php storm seems able to recognise the type of the variable |
|
Interesting, it seems to work for me without the whole namespace if there is an appropriate |
|
Yes, I suggest add full class names including namespaces in DocBlock |
3 participants
Doc block, various error, fixed thanks to incredible new tools..